fix: tweetLimit in community/newsroom isn't been reflected #3138
Conversation
Saumya40-codes
requested review from
derberg,
akshatnema,
magicmatatjahu,
anshgoyalevil,
Mayaleeeee and
asyncapi-bot-eve
as code owners
✅ Deploy Preview for asyncapi-website ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
asyncapi-bot
requested review from
sambhavgupta0705,
BhaswatiRoy,
J0SAL,
TRohit20 and
VaishnaviNandakumar
|
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-3138--asyncapi-website.netlify.app/ |
Do we have any fix for this @Saumya40-codes |
I dont think so, untill we use official twitter apis. I'll still lookout if any other alternatives still exists |
|
@Saumya40-codes Instead of providing strict height to that element, can't we restrict the embed to only fetch 2 tweets there? |
I tried looking for all possible alternatives, but none really were able to work as intended |
|
Close with me |
|
@Saumya40-codes what's the status of this issue? |
|
|
@Saumya40-codes I can't see the tweets even though I'm logged in to twitter 
|
Twitter plugins needs to be inconsistent enough then imo here im using their official site for the same ( one from browser having not logged in to twitter and otherwise)
here is the link for the same: https://publish.twitter.com/?dnt=1&query=%40AsyncAPISpec&widget=Timeline Additionally, this issue is pretty much from twitter plugins only, ref this and this |
|
@Saumya40-codes any update on this issue? |
Nope the ones I stated here looks only to be probable fixes:
Rest I'll be needing some suggestions on those for proceeding |
|
@derberg I think we need your assistance here |
|
Sorry @Saumya40-codes we have moved this issue under bounty program so @akshatnema will be working on this one. |
Description
In community/newsroom page here the amount of tweets which are being loaded to show the timeline is uncontrolled, even though we've mentioned
tweetLimit:2within our code, it turns out that this has been the issue with the main package/library itself and nothing can be done much to solve it.Current issue:
AsyncAPI.Newsroom._.AsyncAPI.Initiative.for.event-driven.APIs.-.Google.Chrome.2024-08-07.20-50-46.mp4
To fix this issue though we can provide a specific height for this particular component and enable scrolling for the rest of the tweets to be seen:
AsyncAPI.Newsroom._.AsyncAPI.Initiative.for.event-driven.APIs.-.Google.Chrome.2024-08-09.22-05-18.mp4
Related issue(s)
Resolves #3132
Additionally, the problem of user who isn't logged in to twitter seeing a message something like below as pointed out by @sambhavgupta0705
Unfortunately this issue exists in twitter's core widget plugin service only ( ref this )